Conversation
gunnarvelle
force-pushed
the
add-published-count
branch
from
8db710b to
60ea360
Compare
|
Som jeg skrev på issuet, vil egentlig en published-count løse problemet de beskriver på trello? |
gunnarvelle
force-pushed
the
add-published-count
branch
from
6895281 to
520c6a1
Compare
amatho
left a comment
amatho
left a comment
There was a problem hiding this comment.
Ser ut som migreringene funker fint, men det mangler logikk for å oppdatere publishedCount ved vanlig publiseringsflyt (både i draft-api og article-api)
| override lazy val whereClause: SQLSyntax = sqls"$columnNameSQL is not null" | ||
|
|
||
| private def countOtherVersions(revision: Int, articleId: Long)(implicit session: DBSession): Long = { | ||
| sql"select count(*) from $tableNameSQL where revision < $revision and article_id = $articleId" |
There was a problem hiding this comment.
Ser at det tar ganske lang tid å kjøre denne lokalt med dump fra test, men ikke sikkert det er verdt bryet å skulle skrive den raskere 😄
There was a problem hiding this comment.
Den er jo avhengig av å telle opp gamle varianter av artikler, så tar jo tid. Kom gjerne med forslag til måter å gjøre den kjappere.
There was a problem hiding this comment.
Virker som det går an å gjøre det med en SQL-only migrering, men det ble ganske syre, så tenker det er fint som det er 😄
draft-api/src/main/scala/no/ndla/draftapi/service/ConverterService.scala
Outdated
Show resolved
Hide resolved
draft-api/src/main/scala/no/ndla/draftapi/service/ConverterService.scala
Outdated
Show resolved
Hide resolved
draft-api/src/main/scala/no/ndla/draftapi/db/migration/V83__SetPublishedCount.scala
Show resolved
Hide resolved
There was a problem hiding this comment.
Kanskje lurt med noen tester på disse migreringene?
There was a problem hiding this comment.
Migreringene er avhengig av nye spørringer, så blir fort litt komplekst. Men kan sikkert mocke noe her. Ser på det.
common/src/main/scala/no/ndla/common/model/domain/draft/Draft.scala
Outdated
Show resolved
Hide resolved
gunnarvelle
force-pushed
the
add-published-count
branch
from
0c230d3 to
977f288
Compare
3 participants
https://github.com/NDLANO/Issues/issues/4493
Legger til publishedCount på draft, article og i search-api.